Skip to main contentSkip to issue filters
to select issuesto navigate
1 / 314issues
10d effort
src/common/Knuth.java
Use "java.util.Random.nextInt()" instead.
  • L36
  • 5min effort
Use "java.util.Random.nextInt()" instead.
  • L54
  • 5min effort
src/common/Queue.java
Rename this generic name to match the regular expression '^[A-Z][0-9]?$'.
  • L26
  • 10min effort
src/common/RandomGenerator.java
Save and re-use this "Random".
  • L15
  • 5min effort
src/common/RandomSet.java
Override the "equals" method in this class.
  • L5
  • 30min effort
src/common/StdIn.java
Do not forget to remove this deprecated code someday.
  • L587
  • 10min effort
Do not forget to remove this deprecated code someday.
  • L601
  • 10min effort
Do not forget to remove this deprecated code someday.
  • L612
  • 10min effort
src/common/StdOut.java
Replace this use of System.out or System.err by a logger.
  • L68
  • 10min effort
Replace charset name argument with StandardCharsets.UTF_8
  • L68
  • 1min effort
Replace this use of System.out or System.err by a logger.
  • L70
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L321
  • 10min effort
src/custom/fattree/Address.java
Rename this field "_1" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L4
  • 2min effort
Rename this field "_2" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L5
  • 2min effort
Rename this field "_3" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L6
  • 2min effort
Rename this field "_4" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L7
  • 2min effort
Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L9
  • 2min effort
Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L9
  • 2min effort
Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L9
  • 2min effort
Rename this local variable to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L9
  • 2min effort
src/custom/fattree/FatTreeFlowClassifier.java
Make flowSizesPerDuration a static final constant or non-public and provide accessors if needed.
  • L13
  • 10min effort
Make outgoingTraffic a static final constant or non-public and provide accessors if needed.
  • L14
  • 10min effort
Make flowTable a static final constant or non-public and provide accessors if needed.
  • L15
  • 10min effort
src/custom/fattree/FatTreeRoutingAlgorithm.java
Add a "clone()" method to this class.
  • L12
  • 30min effort
Make suffixTables a static final constant or non-public and provide accessors if needed.
  • L15
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L137
  • 10min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L257
  • 5min effort
src/custom/smallworld/GridGraph.java
Replace the synchronized class "Stack" by an unsynchronized one such as "Deque".
  • L36
  • 20min effort
Do something with the "boolean" value returned by "createNewFile".
  • L258
  • 15min effort
Use try-with-resources or close this "FileWriter" in a "finally" clause.
  • L260
  • 5min effort
Do something with the "boolean" value returned by "createNewFile".
  • L278
  • 15min effort
Use try-with-resources or close this "FileWriter" in a "finally" clause.
  • L280
  • 5min effort
src/events/DReachingENBEvent.java
Make this "public static type" field final
  • L20
  • 20min effort
Make type a static final constant or non-public and provide accessors if needed.
  • L20
  • 10min effort
Make the enclosing method "static" or remove this set.
  • L67
  • 20min effort
Make the enclosing method "static" or remove this set.
  • L71
  • 20min effort
src/events/EMovingInSwitchEvent.java
Make type a static final constant or non-public and provide accessors if needed.
  • L20
  • 10min effort
src/graph/Coordination.java
Remove this unused "nRow" private field.
  • L6
  • 5min effort
Remove the "nRow" field and declare it as a local variable in the relevant methods.
  • L6
  • 5min effort
Define and throw a dedicated exception instead of using a generic one.
  • L24
  • 20min effort
src/graph/Graph.java
Rename this field "V" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L7
  • 2min effort
Rename this field "E" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L8
  • 2min effort
Rename this method name to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L36
  • 5min effort
Remove this unused method parameter "path".
  • L56
  • 5min effort
Remove this method and declare a constant for this value.
  • L57
  • 5min effort
src/infrastructure/element/Buffer.java
Make physicalLayer a static final constant or non-public and provide accessors if needed.
  • L9
  • 10min effort
src/infrastructure/element/Element.java
Refactor this method to not always return the same value.
  • L54
  • 6min effort
Remove this conditional structure or edit its code blocks so that they're not all the same.
  • L58
  • 15min effort
Replace this if-then-else statement by a single return statement.
  • L58
  • 2min effort
src/infrastructure/element/LimitedBuffer.java
Define a constant instead of duplicating this literal "ERROR: Buffer: " 3 times.
  • L34
  • 8min effort
Define a constant instead of duplicating this literal " oversized" 3 times.
  • L34
  • 8min effort
Replace this use of System.out or System.err by a logger.
  • L34
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L58
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L69
  • 10min effort
src/infrastructure/entity/Node.java
Make physicalLayer a static final constant or non-public and provide accessors if needed.
  • L10
  • 10min effort
Make dataLinkLayer a static final constant or non-public and provide accessors if needed.
  • L11
  • 10min effort
src/infrastructure/event/Event.java
Rename this class.
  • L22
  • 5min effort
Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.
  • L26
  • 2min effort
src/infrastructure/state/State.java
Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.
  • L19
  • 2min effort
Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.
  • L20
  • 2min effort
Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.
  • L22
  • 2min effort
Make element a static final constant or non-public and provide accessors if needed.
  • L23
  • 10min effort
Make type a static final constant or non-public and provide accessors if needed.
  • L24
  • 10min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L111
  • 5min effort
src/javatuples/Decade.java
19 duplicated blocks of code must be removed.
  • 3h20min effort
Make "val0" transient or serializable.
  • L34
  • 30min effort
Make "val1" transient or serializable.
  • L35
  • 30min effort
Make "val2" transient or serializable.
  • L36
  • 30min effort
Make "val3" transient or serializable.
  • L37
  • 30min effort
Make "val4" transient or serializable.
  • L38
  • 30min effort
Make "val5" transient or serializable.
  • L39
  • 30min effort
Make "val6" transient or serializable.
  • L40
  • 30min effort
Make "val7" transient or serializable.
  • L41
  • 30min effort
Make "val8" transient or serializable.
  • L42
  • 30min effort
Make "val9" transient or serializable.
  • L43
  • 30min effort
Method has 10 parameters, which is greater than 7 authorized.
  • L45
  • 20min effort
Refactor this method to reduce its Cognitive Complexity from 29 to the 15 allowed.
  • L115
  • 19min effort
Constructor has 10 parameters, which is greater than 7 authorized.
  • L221
  • 20min effort
src/javatuples/Ennead.java
19 duplicated blocks of code must be removed.
  • 3h20min effort
Make "val0" transient or serializable.
  • L33
  • 30min effort
Make "val1" transient or serializable.
  • L34
  • 30min effort
Make "val2" transient or serializable.
  • L35
  • 30min effort
Make "val3" transient or serializable.
  • L36
  • 30min effort
Make "val4" transient or serializable.
  • L37
  • 30min effort
Make "val5" transient or serializable.
  • L38
  • 30min effort
Make "val6" transient or serializable.
  • L39
  • 30min effort
Make "val7" transient or serializable.
  • L40
  • 30min effort
Make "val8" transient or serializable.
  • L41
  • 30min effort
Method has 9 parameters, which is greater than 7 authorized.
  • L43
  • 20min effort
Refactor this method to reduce its Cognitive Complexity from 27 to the 15 allowed.
  • L113
  • 17min effort
Constructor has 9 parameters, which is greater than 7 authorized.
  • L212
  • 20min effort
src/javatuples/KeyValue.java
3 duplicated blocks of code must be removed.
  • 40min effort
Make "key" transient or serializable.
  • L26
  • 30min effort
Make "value" transient or serializable.
  • L27
  • 30min effort
src/javatuples/LabelValue.java
3 duplicated blocks of code must be removed.
  • 40min effort
Make "label" transient or serializable.
  • L26
  • 30min effort
Make "value" transient or serializable.
  • L27
  • 30min effort
src/javatuples/Octet.java
15 duplicated blocks of code must be removed.
  • 2h40min effort
Make "val0" transient or serializable.
  • L32
  • 30min effort
Make "val1" transient or serializable.
  • L33
  • 30min effort
Make "val2" transient or serializable.
  • L34
  • 30min effort
Make "val3" transient or serializable.
  • L35
  • 30min effort
Make "val4" transient or serializable.
  • L36
  • 30min effort
Make "val5" transient or serializable.
  • L37
  • 30min effort
Make "val6" transient or serializable.
  • L38
  • 30min effort
Make "val7" transient or serializable.
  • L39
  • 30min effort
Method has 8 parameters, which is greater than 7 authorized.
  • L41
  • 20min effort
Refactor this method to reduce its Cognitive Complexity from 25 to the 15 allowed.
  • L109
  • 15min effort
Constructor has 8 parameters, which is greater than 7 authorized.
  • L201
  • 20min effort
src/javatuples/Pair.java
3 duplicated blocks of code must be removed.
  • 40min effort
Make "val0" transient or serializable.
  • L25
  • 30min effort
Make "val1" transient or serializable.
  • L26
  • 30min effort
Method has 8 parameters, which is greater than 7 authorized.
  • L263
  • 20min effort
Method has 8 parameters, which is greater than 7 authorized.
  • L270
  • 20min effort
Method has 8 parameters, which is greater than 7 authorized.
  • L277
  • 20min effort
Method has 8 parameters, which is greater than 7 authorized.
  • L460
  • 20min effort
src/javatuples/Quartet.java
4 duplicated blocks of code must be removed.
  • 50min effort
Make "val0" transient or serializable.
  • L27
  • 30min effort
Make "val1" transient or serializable.
  • L28
  • 30min effort
Make "val2" transient or serializable.
  • L29
  • 30min effort
Make "val3" transient or serializable.
  • L30
  • 30min effort
Refactor this method to reduce its Cognitive Complexity from 17 to the 15 allowed.
  • L99
  • 7min effort
src/javatuples/Quintet.java
6 duplicated blocks of code must be removed.
  • 1h10min effort
Make "val0" transient or serializable.
  • L29
  • 30min effort
Make "val1" transient or serializable.
  • L30
  • 30min effort
Make "val2" transient or serializable.
  • L31
  • 30min effort
Make "val3" transient or serializable.
  • L32
  • 30min effort
Make "val4" transient or serializable.
  • L33
  • 30min effort
Refactor this method to reduce its Cognitive Complexity from 19 to the 15 allowed.
  • L102
  • 9min effort
src/javatuples/Septet.java
11 duplicated blocks of code must be removed.
  • 2h effort
Make "val0" transient or serializable.
  • L31
  • 30min effort
Make "val1" transient or serializable.
  • L32
  • 30min effort
Make "val2" transient or serializable.
  • L33
  • 30min effort
Make "val3" transient or serializable.
  • L34
  • 30min effort
Make "val4" transient or serializable.
  • L35
  • 30min effort
Make "val5" transient or serializable.
  • L36
  • 30min effort
Make "val6" transient or serializable.
  • L37
  • 30min effort
Refactor this method to reduce its Cognitive Complexity from 23 to the 15 allowed.
  • L106
  • 13min effort
src/javatuples/Sextet.java
8 duplicated blocks of code must be removed.
  • 1h30min effort
Make "val0" transient or serializable.
  • L30
  • 30min effort
Make "val1" transient or serializable.
  • L31
  • 30min effort
Make "val2" transient or serializable.
  • L32
  • 30min effort
Make "val3" transient or serializable.
  • L33
  • 30min effort
Make "val4" transient or serializable.
  • L34
  • 30min effort
Make "val5" transient or serializable.
  • L35
  • 30min effort
Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed.
  • L104
  • 11min effort
src/javatuples/Triplet.java
3 duplicated blocks of code must be removed.
  • 40min effort
Make "val0" transient or serializable.
  • L26
  • 30min effort
Make "val1" transient or serializable.
  • L27
  • 30min effort
Make "val2" transient or serializable.
  • L28
  • 30min effort
src/javatuples/Tuple.java
Make "valueArray" transient or serializable.
  • L25
  • 30min effort
Make "valueList" transient or serializable.
  • L26
  • 30min effort
Do not forget to remove this deprecated code someday.
  • L34
  • 10min effort
src/javatuples/Unit.java
1 duplicated blocks of code must be removed.
  • 20min effort
Make "val0" transient or serializable.
  • L24
  • 30min effort
Method has 8 parameters, which is greater than 7 authorized.
  • L219
  • 20min effort
Method has 8 parameters, which is greater than 7 authorized.
  • L226
  • 20min effort
Method has 9 parameters, which is greater than 7 authorized.
  • L233
  • 20min effort
Method has 9 parameters, which is greater than 7 authorized.
  • L240
  • 20min effort
Method has 8 parameters, which is greater than 7 authorized.
  • L396
  • 20min effort
Method has 9 parameters, which is greater than 7 authorized.
  • L407
  • 20min effort
src/network/Topology.java
Make cordOfNodes a static final constant or non-public and provide accessors if needed.
  • L26
  • 10min effort
Make pairGenerator a static final constant or non-public and provide accessors if needed.
  • L29
  • 10min effort
Remove this useless assignment to local variable "sourceNodeIDs".
  • L144
  • 15min effort
Remove this useless assignment to local variable "destinationNodeIDs".
  • L145
  • 15min effort
src/network/elements/EntranceBuffer.java
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L19
  • 5min effort
src/network/elements/ExitBuffer.java
Replace this use of System.out or System.err by a logger.
  • L38
  • 10min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L46
  • 5min effort
src/network/elements/Packet.java
Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.
  • L15
  • 2min effort
Make dataLinkLayer a static final constant or non-public and provide accessors if needed.
  • L17
  • 10min effort
src/network/entities/Host.java
Make type a static final constant or non-public and provide accessors if needed.
  • L12
  • 10min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L19
  • 5min effort
Change this instance-reference to a static reference.
  • L63
  • 5min effort
src/network/entities/Link.java
Use static access with "weightedloadexperiment.pairstrategies.PairGenerator" for "isOversubscriptedLink".
  • L55
  • 5min effort
src/network/entities/Switch.java
Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.
  • L6
  • 2min effort
src/network/entities/TypeOfHost.java
Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.
  • L4
  • 2min effort
Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.
  • L4
  • 2min effort
Rename this constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'.
  • L4
  • 2min effort
src/network/layers/DataLinkLayer.java
Make packet a static final constant or non-public and provide accessors if needed.
  • L6
  • 10min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L13
  • 5min effort
src/network/layers/Layer.java
Make routingAlgorithm a static final constant or non-public and provide accessors if needed.
  • L7
  • 10min effort
src/network/layers/NetworkLayer.java
Replace this use of System.out or System.err by a logger.
  • L63
  • 10min effort
src/network/layers/PhysicalLayer.java
Make exitBuffers a static final constant or non-public and provide accessors if needed.
  • L12
  • 10min effort
The type of the "exitBuffers" object should be an interface such as "Map" rather than the implementation "HashMap".
  • L12
  • 10min effort
Make entranceBuffers a static final constant or non-public and provide accessors if needed.
  • L13
  • 10min effort
The type of the "entranceBuffers" object should be an interface such as "Map" rather than the implementation "HashMap".
  • L13
  • 10min effort
Make sourceQueue a static final constant or non-public and provide accessors if needed.
  • L14
  • 10min effort
Make links a static final constant or non-public and provide accessors if needed.
  • L15
  • 10min effort
The type of the "links" object should be an interface such as "Map" rather than the implementation "HashMap".
  • L15
  • 10min effort
Make simulator a static final constant or non-public and provide accessors if needed.
  • L16
  • 10min effort
Make node a static final constant or non-public and provide accessors if needed.
  • L17
  • 10min effort
Remove this unused method parameter "k".
  • L34
  • 5min effort
src/network/states/enb/N0.java
Complete the task associated to this TODO comment.
  • L31
Merge this if statement with the enclosing one.
  • L32
  • 5min effort
src/network/states/enb/N1.java
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L13
  • 5min effort
src/network/states/packet/StateP1.java
Make packet a static final constant or non-public and provide accessors if needed.
  • L10
  • 10min effort
Remove this unused method parameter "ev".
  • L12
  • 5min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L18
  • 5min effort
src/network/states/packet/StateP2.java
Make packet a static final constant or non-public and provide accessors if needed.
  • L10
  • 10min effort
Remove this unused method parameter "ev".
  • L12
  • 5min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L18
  • 5min effort
src/network/states/packet/StateP3.java
Make packet a static final constant or non-public and provide accessors if needed.
  • L11
  • 10min effort
Remove this unused method parameter "ev".
  • L13
  • 5min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L19
  • 5min effort
src/network/states/packet/StateP4.java
Make packet a static final constant or non-public and provide accessors if needed.
  • L11
  • 10min effort
Remove this unused method parameter "ev".
  • L13
  • 5min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L19
  • 5min effort
src/network/states/packet/StateP5.java
Make packet a static final constant or non-public and provide accessors if needed.
  • L11
  • 10min effort
Remove this unused method parameter "ev".
  • L13
  • 5min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L19
  • 5min effort
src/network/states/packet/StateP6.java
Make packet a static final constant or non-public and provide accessors if needed.
  • L10
  • 10min effort
Remove this unused method parameter "ev".
  • L12
  • 5min effort
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L18
  • 5min effort
src/network/states/sourcequeue/Sq2.java
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L13
  • 5min effort
src/.../states/unidirectionalway/W1.java
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L14
  • 5min effort
src/.../states/unidirectionalway/W2.java
Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
  • L14
  • 5min effort
src/simulator/DiscreteEventSimulator.java
Make numReceived a static final constant or non-public and provide accessors if needed.
  • L10
  • 10min effort
Make receivedPacketPerUnit a static final constant or non-public and provide accessors if needed.
  • L11
  • 10min effort
Move the array designator from the variable to the type.
  • L11
  • 5min effort
Make numSent a static final constant or non-public and provide accessors if needed.
  • L12
  • 10min effort
Make numLoss a static final constant or non-public and provide accessors if needed.
  • L13
  • 10min effort
Make totalPacketTime a static final constant or non-public and provide accessors if needed.
  • L14
  • 10min effort
Make numEvent a static final constant or non-public and provide accessors if needed.
  • L15
  • 10min effort
Make totalHop a static final constant or non-public and provide accessors if needed.
  • L20
  • 10min effort
Make sizeOfCurrEvents a static final constant or non-public and provide accessors if needed.
  • L22
  • 10min effort
Make halfSizeOfEvents a static final constant or non-public and provide accessors if needed.
  • L26
  • 10min effort
Rename this field "IS_LIMIT" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L28
  • 2min effort
Rename field "VERBOSE" to prevent any misunderstanding/clash with field "verbose" defined on line 18.
  • L29
  • 10min effort
Rename this field "VERBOSE" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L29
  • 2min effort
Rename this field "TIME_LIMIT" to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L30
  • 2min effort
Rename this method name to match the regular expression '^[a-z][a-zA-Z0-9]*$'.
  • L32
  • 5min effort
Replace this use of System.out or System.err by a logger.
  • L87
  • 10min effort
Introduce a new variable instead of reusing the parameter "ev".
  • L92
  • 5min effort
Remove this unused method parameter "currentTime".
  • L135
  • 5min effort
src/simulator/Simulator.java
Rename this class.
  • L9
  • 5min effort
Make topology a static final constant or non-public and provide accessors if needed.
  • L14
  • 10min effort
Make currentEvents a static final constant or non-public and provide accessors if needed.
  • L16
  • 10min effort
src/weightedloadexperiment/GraphPanel.java
Remove these useless parentheses.
  • L86
  • 1min effort
Save and re-use this "Random".
  • L187
  • 5min effort
Use static access with "javax.swing.WindowConstants" for "EXIT_ON_CLOSE".
  • L196
  • 5min effort
Make this anonymous inner class a lambda (sonar.java.source not set. Assuming 8 or greater.)
  • L204
  • 5min effort
Use static access with "javax.swing.WindowConstants" for "EXIT_ON_CLOSE".
  • L216
  • 5min effort
src/weightedloadexperiment/ThroughputExperiment.java
Replace this use of System.out or System.err by a logger.
  • L37
  • 10min effort
Remove this unused "count" local variable.
  • L42
  • 5min effort
Iterate over the "entrySet" instead of the "keySet".
  • L43
  • 5min effort
Remove this useless assignment to local variable "rawThroughput".
  • L77
  • 15min effort
Remove this unused "rawThroughput" local variable.
  • L77
  • 5min effort
Cast one of the operands of this division operation to a "double".
  • L79
  • 5min effort
Remove this useless assignment to local variable "alternativeRawThroughput".
  • L80
  • 15min effort
Remove this unnecessary cast to "double".
  • L80
  • 5min effort
Replace this use of System.out or System.err by a logger.
  • L84
  • 10min effort
Remove this unused "rxPacket" local variable.
  • L114
  • 5min effort
Remove this unused "thp" local variable.
  • L115
  • 5min effort
Replace this use of System.out or System.err by a logger.
  • L133
  • 10min effort
Iterate over the "entrySet" instead of the "keySet".
  • L139
  • 5min effort
Replace this use of System.out or System.err by a logger.
  • L140
  • 10min effort
This block of commented-out lines of code should be removed.
  • L148
  • 5min effort
This block of commented-out lines of code should be removed.
  • L150
  • 5min effort
This block of commented-out lines of code should be removed.
  • L156
  • 5min effort
This block of commented-out lines of code should be removed.
  • L162
  • 5min effort
src/.../pairstrategies/ForcePair.java
2 duplicated blocks of code must be removed.
  • 30min effort
"modulo" is the name of a field in "OverSubscription".
  • L10
  • 5min effort
Replace this use of System.out or System.err by a logger.
  • L30
  • 10min effort
src/.../pairstrategies/InterPodIncoming.java
1 duplicated blocks of code must be removed.
  • 20min effort
Remove the "adjMx" field and declare it as a local variable in the relevant methods.
  • L16
  • 5min effort
Remove this expression which always evaluates to "true"
  • L88
  • 10min effort
Remove this expression which always evaluates to "true"
  • L97
  • 10min effort
Method has 8 parameters, which is greater than 7 authorized.
  • L104
  • 20min effort
Introduce a new variable instead of reusing the parameter "expectedSrc".
  • L107
  • 5min effort
Remove this useless assignment to local variable "found".
  • L109
  • 15min effort
Introduce a new variable instead of reusing the parameter "found".
  • L109
  • 5min effort
Remove this useless assignment to local variable "currPod".
  • L113
  • 15min effort
Introduce a new variable instead of reusing the parameter "currPod".
  • L113
  • 5min effort
Remove this useless assignment to local variable "currPod".
  • L115
  • 15min effort
Introduce a new variable instead of reusing the parameter "currPod".
  • L115
  • 5min effort
Remove this useless assignment to local variable "found".
  • L126
  • 15min effort
Introduce a new variable instead of reusing the parameter "found".
  • L126
  • 5min effort
Remove this useless assignment to local variable "currPod".
  • L134
  • 15min effort
Introduce a new variable instead of reusing the parameter "currPod".
  • L134
  • 5min effort
Remove this useless assignment to local variable "currPod".
  • L136
  • 15min effort
Introduce a new variable instead of reusing the parameter "currPod".
  • L136
  • 5min effort
Remove this unused "equal" local variable.
  • L154
  • 5min effort
Iterate over the "entrySet" instead of the "keySet".
  • L155
  • 5min effort
Replace this use of System.out or System.err by a logger.
  • L173
  • 10min effort
Introduce a new variable instead of reusing the parameter "realCore".
  • L176
  • 5min effort
Replace this use of System.out or System.err by a logger.
  • L177
  • 10min effort
Introduce a new variable instead of reusing the parameter "realCore".
  • L198
  • 5min effort
Replace this use of System.out or System.err by a logger.
  • L199
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L209
  • 10min effort
src/.../pairstrategies/OverSubscription.java
1 duplicated blocks of code must be removed.
  • 20min effort
Make modulo a static final constant or non-public and provide accessors if needed.
  • L18
  • 10min effort
Make k a static final constant or non-public and provide accessors if needed.
  • L20
  • 10min effort
Make routing a static final constant or non-public and provide accessors if needed.
  • L22
  • 10min effort
Make graph a static final constant or non-public and provide accessors if needed.
  • L23
  • 10min effort
Rename "k" which hides the field declared at line 20.
  • L40
  • 5min effort
Remove this unused method parameter "destination".
  • L62
  • 5min effort
src/.../pairstrategies/SameIDOutgoing.java
Refactor this method to reduce its Cognitive Complexity from 20 to the 15 allowed.
  • L55
  • 10min effort
Reduce the total number of break and continue statements in this loop to use at most one.
  • L61
  • 20min effort
Remove this useless assignment to local variable "i".
  • L97
  • 15min effort
Replace this use of System.out or System.err by a logger.
  • L98
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L103
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L106
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L107
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L109
  • 10min effort
Remove this unused private "isSameSubNet" method.
  • L126
  • 5min effort
Replace this if-then-else statement by a single return statement.
  • L130
  • 2min effort
Replace this use of System.out or System.err by a logger.
  • L183
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L186
  • 10min effort
Replace this use of System.out or System.err by a logger.
  • L195
  • 10min effort
src/.../pairstrategies/interpod/MinimalCoreSwitches.java
Remove the "coreInPath" field and declare it as a local variable in the relevant methods.
  • L12
  • 5min effort
Remove this useless assignment to local variable "usedPods".
  • L56
  • 15min effort
Remove this expression which always evaluates to "true"
  • L95
  • 10min effort
Remove this expression which always evaluates to "true"
  • L109
  • 10min effort
Replace this if-then-else statement by a single return statement.
  • L129
  • 2min effort
314 of 314 shown